Skip to content

feat: Add removeKey util function#29

Merged
klmhyeonwoo merged 2 commits into
mainfrom
feature/util-workspace
Sep 3, 2025
Merged

feat: Add removeKey util function#29
klmhyeonwoo merged 2 commits into
mainfrom
feature/util-workspace

Conversation

@klmhyeonwoo
Copy link
Copy Markdown
Member

@klmhyeonwoo klmhyeonwoo commented Sep 2, 2025

Description

  • removeKey 유틸함수가 추가되었어요.
objectUtil.removeKey("b", { a: 1, b: 2, c: 3 }); // { a: 1, c: 3 }

closes: #30

@klmhyeonwoo
Copy link
Copy Markdown
Member Author

klmhyeonwoo commented Sep 2, 2025

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 68.92% 122 / 177
🔵 Statements 68.92% 122 / 177
🔵 Functions 94.44% 17 / 18
🔵 Branches 91.42% 64 / 70
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
package/objectUtil/removeKey/index.ts 100% 100% 100% 100%
Generated in workflow #46 for commit ba9ea9c by the Vitest Coverage Report Action

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a new removeKey utility function to the objectUtil module that creates a new object with a specified key removed from the input object.

  • Implements a removeKey function using object destructuring and rest syntax
  • Adds comprehensive test coverage for various scenarios
  • Updates module exports and documentation

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
package/objectUtil/removeKey/index.ts Core implementation of the removeKey function
package/objectUtil/removeKey/index.test.ts Test suite covering key removal scenarios
package/objectUtil/index.ts Export declaration for the new removeKey function
README.md Documentation and usage example for removeKey

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread package/objectUtil/removeKey/index.test.ts Outdated
Comment thread package/objectUtil/removeKey/index.test.ts Outdated
Comment thread package/objectUtil/removeKey/index.test.ts Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Member

@prgmr99 prgmr99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다~~!!

@klmhyeonwoo klmhyeonwoo merged commit edef4a4 into main Sep 3, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

objectUtil > removeKey()

3 participants